PR Summary

Low Risk
Low risk: changes are limited to emitting additional PostHog analytics events and removing one noisy event, with no functional business-logic changes beyond extra side-effect calls.

Overview
Adds new PostHog tracking across key user actions: folder create/delete, scheduled task create/delete, workflow import/export, logs filter application, search result selections, and knowledge base document deletion (including workspace group attribution where available).

Reduces analytics noise by removing the task_marked_read event and updating task_message_sent to always include workspace_id; the PostHog event catalog (events.ts) is expanded to type these new events/properties.

^{Reviewed by Cursor Bugbot for commit 8858f57. Configure here.}

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 4, 2026 11:49pm

Greptile Summary

This PR is a PostHog analytics audit that removes noisy events and adds 13 new, targeted event captures across server-side API routes and client-side components. The changes span folder lifecycle events, scheduled task CRUD, knowledge-base document deletion, task/chat operations, log filter interactions, search result selection, and workflow import/export. The event catalog (events.ts) is updated as a single source of truth, and all new event types have corresponding capture calls verified in the codebase.

Key changes:

• New server events: folder_created, folder_deleted, scheduled_task_created, scheduled_task_deleted, knowledge_base_document_deleted, task_renamed, task_deleted, task_marked_unread — all captured in their respective API routes.
• New client events: task_message_sent, logs_filter_applied (5 filter types), search_result_selected (12 result types), workflow_imported, workflow_exported — captured using the posthogRef ref pattern with usePostHog.
• Pattern consistency: All new client-side captures correctly use an inline posthogRef.current = posthog update inside useCallback closures — except home.tsx which uses a useEffect to update the ref instead, diverging from the pattern established throughout this PR.

Confidence Score: 5/5

Safe to merge — all new analytics captures are fire-and-forget and cannot affect functional behavior.

The sole finding is a P2 style inconsistency in home.tsx (useEffect vs. inline ref update). All event captures are correctly typed, guarded against missing session/workspace data, and consistent with the shared PostHogEventMap. No logic bugs, data-loss risks, or security issues identified.

apps/sim/app/workspace/[workspaceId]/home/home.tsx — minor posthogRef update pattern inconsistency.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph Server["Server-side API Routes"]
        A["POST /api/folders"] -->|folder_created| PH
        B["DELETE /api/folders/:id"] -->|folder_deleted| PH
        C["DELETE /api/knowledge/:id/documents/:docId"] -->|knowledge_base_document_deleted| PH
        D["POST /api/schedules"] -->|scheduled_task_created| PH
        E["DELETE /api/schedules/:id"] -->|scheduled_task_deleted| PH
        F["PATCH /api/mothership/chats/:chatId"] -->|task_renamed or task_marked_unread| PH
        G["DELETE /api/mothership/chats/:chatId"] -->|task_deleted| PH
    end
    subgraph Client["Client-side Components and Hooks"]
        H["home.tsx handleSubmit"] -->|task_message_sent| PH
        I["logs-toolbar.tsx filter handlers"] -->|logs_filter_applied| PH
        J["search-modal.tsx select handlers"] -->|search_result_selected| PH
        K["use-export-workflow.ts"] -->|workflow_exported| PH
        L["use-import-workflow.ts"] -->|workflow_imported| PH
    end
    PH["PostHog Analytics"]

_{Reviews (1): Last reviewed commit: "fix(analytics): correct format field log..." | Re-trigger Greptile}